Skip to content

Version 7.3.0#273

Merged
ildyria merged 4 commits intomasterfrom
version-7.3.0
Jan 28, 2026
Merged

Version 7.3.0#273
ildyria merged 4 commits intomasterfrom
version-7.3.0

Conversation

@ildyria
Copy link
Member

@ildyria ildyria commented Jan 27, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features
    • Enterprise LDAP/Active Directory user authentication (Supporter Edition)
    • Photo details drawer now opens by default
  • Bug Fixes
    • Hotfix: Docker image now includes php-ldap dependency
    • Various stability improvements and migration fixes
  • Documentation
    • Troubleshooting note for LDAP/php-ldap errors after updating
    • Installation requirements updated to list ldap PHP extension
    • Expanded contributor guidelines and album rights notes
  • Other
    • Release announcement updated to v7.3.1

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 27, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adds releases v7.3.1 (hotfix) and v7.3.0 (feature) to release notes and roadmap, updates announcement to v7.3.1, increments total releases to 112, adds PHP ldap to installation requirements, and inserts a troubleshooting entry about php-ldap / facade runtime error (duplicated in the FAQ).

Changes

Cohort / File(s) Summary
Release notes
docs/releases.md		 Inserted v7.3.1 (hotfix) and v7.3.0 (feature) sections; expanded v7.3.0 "Most notable changes"; removed two lines from the v7.2.1 subsection.
Site pages / UI
src/components/widgets/Announcement.astro, src/pages/roadmap.astro		 Updated announcement text/link to v7.3.1; added v7.3.1 and v7.3.0 entries to the roadmap and updated dashboard stats (Latest Version -> 7.3.1, Total Releases -> 112).
Troubleshooting / FAQs
docs/faq_troubleshooting.md		 Added troubleshooting subsection about "PHP Fatal error: ... A facade root has not been set" explaining php-ldap is required; the entry was added in two locations (duplicated).
Installation requirements
docs/installation.md		 Added ldap (php-ldap) to the PHP extensions server requirements list (new since 7.3).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐇 I hopped through changelogs, bright and spry,
Two releases baked beneath the sky,
I nudged in ldap so things run neat,
A tiny fix made builds complete,
I twitch my nose and tap my feet.

🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai bot requested changes Jan 28, 2026
View reviewed changes
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

ildyria and others added 2 commits January 28, 2026 12:25
@ildyria @coderabbitai
Update docs/faq_troubleshooting.md
f35ca7f 
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@ildyria @coderabbitai
Update docs/installation.md
773e99d 
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@ildyria ildyria merged commit 60157c1 into master Jan 28, 2026
1 check passed
@ildyria ildyria deleted the version-7.3.0 branch January 28, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ildyria